Repository navigation
Present Quick Undo and Redo as governed revision history - #264
Merged
Merged
Conversation
mberrys
marked this pull request as ready for review
October 11, 2026 05:03
Keep the governed-history publication path and dev's signature-invalidation acknowledgement together: the Action List run worker takes the publication source path followed by the acknowledgement flag, and the editor host passes both. The rollback summary member stays removed, since governed history replaces it. The worker unit test passes an empty publication path before its acknowledgement argument. Co-Authored-By: Claude Haiku 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Lqo8EDq3w2nU1zekEZDoZc
Co-Authored-By: Claude Haiku 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Lqo8EDq3w2nU1zekEZDoZc
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Quick Undo and Redo now navigate governed revision history. Undo of a published correction runs Core's forward rollback gateway and creates a new PDF; reopening reconstructs Undo/Redo from the retained chain. The Fix workspace shows revision digests, approvals, and publication receipts, with confirmation bound to the displayed revision.
Rollback closes the GUI history connection before dispatching the worker, so the worker can snapshot a checkpointed history. Editor Action List publications retain unique sibling PDFs and carry verified history/artifacts into each new sidecar. A new correction clears Redo. Failed revalidation, stale confirmation, cancellation, and stale completion preserve approved source revisions and receipts.
The approved Core extension accepts a retained original input only when it matches the input of the accepted execution being undone and the request's current digest matches that execution's output. Existing request types and persistence schemas remain unchanged.
Closes #124.
Validation and handoff:
1be5f5a7ce0af39afbc7c3933d921519126630c7; basedevatb08e218360306665f86931b311e641884f818925.--repeat until-fail:2, 122.90 s).d1722e46610ede47bcecbf91d74b8a28a3d36d78b5cac4f3ca793b78f8f823d9; published/Redo09530d36fa3774bee386a50f099ccb30a034fb30e2c8fe21973dacf3cd642f3f; reopened chain contained 6 events before further multi-correction checks.UnitTestsProductOperatorLooppassed and left the checkout clean. Architecture bindings, the six identity-contract tests, seven onboarding tests, and two governed-parity script tests passed.python scripts/agent/check-change.py --base origin/dev --build-dir <approved-external-build> --report <external-report>.UnitTestsOperationHistory::rollbackToOriginalRequiresTheAcceptedExecutionBeingUndoneandUnitTestsEditorHost::fixJourneyPublishesOnlyAnApprovedPlanBoundToTheDisplayedIdentity, plusfixRollbackReturnsToARecordedRevision. They cover publish/rollback/reopen, multiple Undo/Redo transitions, a new correction clearing Redo, stale confirmation, refused revalidation, tampered input, and retained approved events/receipts.ProductQuickAccessibilitySmoke --preview-fixtures loop-preflight/testdata/fixturespassed with offscreen/software rendering. This does not qualify installed packages or native accessibility./bin/bashthrough WSL; the corresponding Linux policy run passed all 467 tests (8 declared skips).unresolved; installed package/native accessibility qualification is not asserted. The full Linux/Windows qualification jobs run only for stable pushes or manual dispatch and are skipped on this PR; independent-integrity requires a separate dispatch with oracle inputs.Anti-slop review: removed shared-directory deletion/helpers and obsolete filesystem-publication commentary. Fixtures pass their owned recipe directories through the internal constructors, with no global identity mutation or test-only production wrapper. Identity, approval, cancellation, provenance, and failure guards remain in place.
Need help on this PR? Tag
@codesmith-botwith what you need. Autofix is disabled.